Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add LLO wide channels integration test #16587

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Add LLO wide channels integration test #16587

wants to merge 2 commits into from
+302 −0

Conversation

cawthorne
Copy link
Contributor

Run:
go test -run TestIntegration_LLO_stress_test_with_wide_channels_and_transmit_errors ./core/services/ocr2/plugins/llo

@cawthorne cawthorne requested review from a team as code owners February 26, 2025 13:41
@cawthorne cawthorne requested a review from Atrax1 February 26, 2025 13:41
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Feb 26, 2025 •
edited
Loading

AER Report: CI Core

aer_workflow , commit , Clean Go Tidy & Generate , Detect Changes , Scheduled Run Frequency , GolangCI Lint (.) , Core Tests (go_core_tests) , Core Tests (go_core_tests_integration) , Core Tests (go_core_ccip_deployment_tests) , Core Tests (go_core_fuzz) , Core Tests (go_core_race_tests) , test-scripts , lint , SonarQube Scan

1. Unnecessary conversion error: lint

Source of Error: 
core/services/ocr2/plugins/llo/integration_test.go:1183:30: unnecessary conversion (unconvert)
					ExpirationWindow: uint32(expirationWindow),
					 ^
**Why**: The error is caused by an unnecessary type conversion in the code. The variable `expirationWindow` is already of type `uint32`, so converting it again to `uint32` is redundant.

Suggested fix: Remove the redundant type conversion. Change uint32(expirationWindow) to just expirationWindow.

AER Report: Operator UI CI ran successfully ✅

aer_workflow , commit

@github-actions GitHub Actions
Copy link
Contributor

Flakeguard Summary

Ran new or updated tests between develop and 52b8df6 (feature/lllo-wide-channels-integration-test).

View Flaky Detector Details | Compare Changes

Found Flaky Tests ❌

1 Results
Name Pass Ratio Panicked? Timed Out? Race? Runs Successes Failures Skips Package Package Panicked? Avg Duration Code Owners
TestIntegration_LLO_stress_test_with_wide_channels_and_transmit_errors 0% true true false 3 0 3 0 github.com/smartcontractkit/chainlink/v2/core/services/ocr2/plugins/llo true 0s Unknown

Artifacts

For detailed logs of the failed tests, please refer to the artifact failed-test-results-with-logs.json.

@cl-sonarqube-production
Copy link

Quality Gate failed Quality Gate failed

Failed conditions
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube

Catch issues before they fail your Quality Gate with our IDE extension SonarQube IDE SonarQube IDE

@github-actions GitHub Actions
Copy link
Contributor

Flakeguard Summary

Ran new or updated tests between develop and 7296c32 (feature/lllo-wide-channels-integration-test).

View Flaky Detector Details | Compare Changes

Found Flaky Tests ❌

1 Results
Name Pass Ratio Panicked? Timed Out? Race? Runs Successes Failures Skips Package Package Panicked? Avg Duration Code Owners
TestIntegration_LLO_stress_test_with_wide_channels_and_transmit_errors 0% true true false 3 0 3 0 github.com/smartcontractkit/chainlink/v2/core/services/ocr2/plugins/llo true 0s Unknown

Artifacts

For detailed logs of the failed tests, please refer to the artifact failed-test-results-with-logs.json.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Atrax1 Atrax1 Awaiting requested review from Atrax1 Atrax1 is a code owner automatically assigned from smartcontractkit/foundations

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@cawthorne